New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Guide multi app solution #1085

Merged

standeren merged 46 commits into master from guide-multi-app-solution

Nov 24, 2023

Contributor

standeren commented Aug 24, 2023

Description

Related Issue(s)

#{issue number}

Verification

Your code builds clean without any errors or warnings
Manual testing done (required)
Relevant automated test added (if you find this hard, leave it and we'll help out)
All tests run green

Documentation

User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

standeren force-pushed the guide-multi-app-solution branch 3 times, most recently from b716525 to f351691 Compare

August 24, 2023 14:21

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/_index.en.md Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/preparations/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/preparations/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/preparations/_index.en.md Outdated Show resolved Hide resolved

standeren force-pushed the guide-multi-app-solution branch 2 times, most recently from d314b0b to 29f1097 Compare

September 14, 2023 11:27

standeren force-pushed the guide-multi-app-solution branch 5 times, most recently from 4a0e96b to 6086e86 Compare

September 29, 2023 13:58

acn-sbuad reviewed

View reviewed changes

content/app/guides/maskinporten-app-integration/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/maskinporten-app-integration/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/testing/_index.en.md Outdated Show resolved Hide resolved

acn-sbuad reviewed

View reviewed changes

content/app/guides/multi-app-solution/testing/_index.en.md Outdated Show resolved Hide resolved

standeren added 5 commits

November 6, 2023 14:52


          Add folder for multi-app-solution documentation

0725b08


          Add guide for multi-app-solution

9c20ddd


          Fix links and add conditional version demand

611675a


          Add testing docs

5f5f332


          Add links to policy

5b5160c

Contributor

acn-sbuad commented Nov 7, 2023 •

edited

Loading

General feedback: go over and be consistent in the referencing of application A and B.
Is it Application A or application A?

acn-sbuad and others added 4 commits

November 7, 2023 16:49


          minor fixes to instrunction for app b

ede0f23


          Fix comments from review

c24aa6f


          Add expandable maskinporten guide in app-mp-integration guide

b2439dd


          Merge branch 'master' into guide-multi-app-solution

ae44788

nkylstad reviewed

View reviewed changes

Member

nkylstad left a comment

Overall looks great! I added some suggestions/comments for you to consider. Would also like the norwegian translations here 😇

content/app/guides/multi-app-solution/_index.nb.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/_index.en.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/instructions/app-a/_index.en.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/instructions/app-b/_index.en.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/multi-app-architecture.drawio.svg Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/testing/_index.en.md Outdated Show resolved Hide resolved

content/app/guides/multi-app-solution/testing/_index.en.md Show resolved Hide resolved

standeren and others added 15 commits

November 22, 2023 14:43


          Change formulation in prerequisites for instructions

8568b66


          Update content/app/guides/multi-app-solution/_index.en.md

16d9ea4

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/_index.en.md

c972ae7

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/_index.en.md

a1f43cc

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/_index.en.md

4c21f0c

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/instructions/app-a/_inde…

8a4399d

…x.en.md

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/_index.en.md

4d42702

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/_index.en.md

4f140be

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/testing/_index.en.md

3909ea9

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/_index.en.md

1187b4c

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/_index.en.md

cac3b1a

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/instructions/app-a/_inde…

bcb0be0

…x.en.md

Co-authored-by: Nina Kylstad <[email protected]>


          Update content/app/guides/multi-app-solution/instructions/app-a/_inde…

0ead7fa

…x.en.md

Co-authored-by: Nina Kylstad <[email protected]>


          Merge branch 'master' into guide-multi-app-solution

2151b02


          Fix PR comments

67269b1

standeren force-pushed the guide-multi-app-solution branch 3 times, most recently from 8e44683 to f03d685 Compare

November 22, 2023 16:19


          Add norwegaian translation for maskinporten app integration

9d8eea2

standeren force-pushed the guide-multi-app-solution branch from f03d685 to 9d8eea2 Compare

November 22, 2023 16:28


          Merge branch 'master' into guide-multi-app-solution

fa15fd8

nkylstad approved these changes

View reviewed changes

Member

nkylstad left a comment

LGTM! Godt jobbet 🥳

standeren merged commit ea666cb into master

1 check passed

standeren deleted the guide-multi-app-solution branch

November 24, 2023 08:39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet